docs: slim CLAUDE.md in favor of global agent-tools skills#98
docs: slim CLAUDE.md in favor of global agent-tools skills#98alex-mextner wants to merge 1 commit into
Conversation
Replace portable dev-discipline sections (foundational rules, naming, comment hygiene, systematic debugging, generic TDD/test discipline, version-control basics, session self-check) with one-line pointers to the now-public skills at github.com/alex-mextner/agent-tools. Keep every ExpenseSyncBot-specific rule: architecture, file paths, commands, deploy, the bun test-runner/:memory:/mock.module plumbing, the knip+codex commit gate, and the project callback_data guidance. Also removes Cyrillic from the Session Wrap-Up section (now a skill pointer), satisfying the English-only repo-docs rule. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
🤖 Stage bot deployed! Test it: https://t.me/ExpenseSyncStageBot Branch: |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 42fee3bf5f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - **Deferred findings**: when skipping a review finding (out of scope, pre-existing), create a GitHub | ||
| issue for it. Don't silently drop known issues. | ||
| > Portable: global skills `atomic-commits`, `pre-commit-gate`, `ai-review-before-commit`, | ||
| > `push-regularly`, `deferred-findings-tracking`, `worktree-isolation`. (No blind `git add -A`; |
There was a problem hiding this comment.
The referenced replacement skill does not exist in github.com/alex-mextner/agent-tools/skills/universal; the available worktree-related skill is worktree-base-trap, while this new pointer says worktree-isolation. Because the old inline worktree workflow was removed in this commit, agents following this guide in a worktree session will fail to load the intended rule and lose the protection against pushing directly to main or touching the main checkout.
Useful? React with 👍 / 👎.
1 participant
Slims the project guide by replacing portable dev-discipline sections with one-line pointers to the now-public skills at github.com/alex-mextner/agent-tools, while keeping every ExpenseSyncBot-specific rule.
Removed → referenced as skills
smallest-change,yagni-kiss-dry,systematic-debuggingsmallest-change,comment-hygiene,dependency-version-ranges,no-type-escape-hatches,unused-params,no-silent-fallbacks,fail-closed-security,atomic-db-transactionsnaming,comment-hygiene,file-header-commentssystematic-debuggingtdd-red-first,test-disciplineatomic-commits,pre-commit-gate,ai-review-before-commit,push-regularly,deferred-findings-tracking,worktree-isolationtask-completion-selfcheck(also drops the Cyrillic, satisfying English-only repo docs)Kept (project-specific)
Architecture, file paths, commands, deploy, the bun test-runner /
:memory:/mock.moduleplumbing, pino logger conventions, biome/lefthook config, the knip+codex commit gate, Telegramcallback_data(fitCallbackData) guidance, bot identity/tone, ZenPlugins workflow.Net: 41 insertions / 62 deletions, one file.
🤖 Generated with Claude Code